Skip to content

Conversation

@kozlov721
Copy link
Collaborator

Purpose

More useful error message.

Specification

Raising an error when filter_task_names contain task names not associated with the dataset

Dependencies & Potential Impact

None / not applicable

Deployment Plan

None / not applicable

Testing & Validation

None / not applicable

@kozlov721 kozlov721 requested a review from a team as a code owner October 3, 2025 18:21
@kozlov721 kozlov721 requested review from conorsim, klemen1999 and tersekmatija and removed request for a team October 3, 2025 18:21
@github-actions github-actions bot added the fix Fixing a bug label Oct 3, 2025
@kozlov721 kozlov721 changed the title Fix/error on wrong task Better error message on missing tasks Oct 3, 2025
@kozlov721 kozlov721 changed the title Better error message on missing tasks Better error message on missing task names Oct 3, 2025
@codecov
Copy link

codecov bot commented Oct 3, 2025

Codecov Report

❌ Patch coverage is 28.57143% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.24%. Comparing base (7f15cba) to head (e09b590).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
luxonis_ml/data/loaders/luxonis_loader.py 28.57% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #369      +/-   ##
==========================================
- Coverage   95.31%   95.24%   -0.08%     
==========================================
  Files         105      105              
  Lines        6535     6540       +5     
==========================================
  Hits         6229     6229              
- Misses        306      311       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kozlov721 kozlov721 merged commit 8a168f0 into main Oct 7, 2025
13 of 15 checks passed
@kozlov721 kozlov721 deleted the fix/error-on-wrong-task branch October 7, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Fixing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants